Skip to content

slowly deprecate skip_event#291

Merged
truthbk merged 1 commit intoDataDog:masterfrom
flyinprogrammer:help_deprecate_skip_event
Mar 7, 2017
Merged

slowly deprecate skip_event#291
truthbk merged 1 commit intoDataDog:masterfrom
flyinprogrammer:help_deprecate_skip_event

Conversation

@flyinprogrammer
Copy link
Copy Markdown
Contributor

We should default this to true since ya'll have been trying to deprecate this feature since 5.3

https://github.com/DataDog/dd-agent/blob/master/checks/network_checks.py#L178-L180

@truthbk truthbk added this to the 1.10.0 milestone Mar 3, 2017
@truthbk
Copy link
Copy Markdown
Member

truthbk commented Mar 3, 2017

Yeah, you're probably right! Thank you so much, as always @flyinprogrammer! Could you rebase to the latest master? The CI broke for a while, the fixes are now in place. Just want to make sure all spec tests are 💚

@flyinprogrammer
Copy link
Copy Markdown
Contributor Author

should be all set now 😄

Copy link
Copy Markdown
Member

@truthbk truthbk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!! :shipit:

@truthbk truthbk merged commit f308bff into DataDog:master Mar 7, 2017
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Jan 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants